-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chef bigquery #507
Chef bigquery #507
Conversation
f14ec17
to
1e12626
Compare
I am a robot that works on MagicModules PRs! I built this PR into one or more PRs on other repositories, and when those are closed, this PR will also be merged and closed. |
I am a robot that works on MagicModules PRs! Once this PR is approved, you can feel free to merge it without taking any further steps. |
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit d5f4294) have been included in your existing downstream PRs. |
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit c35da01) have been included in your existing downstream PRs. |
This looks great! Just have to add the Concourse piece now |
@ndmckinley please take a look at pipeline changes! We'll push this up to Seven after your +2 |
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit 10b126e) have been included in your existing downstream PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This submodule addition should work. :)
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit 2bdf2f5) have been included in your existing downstream PRs. |
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit e55a005) have been included in your existing downstream PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merge LGTM.
@ndmckinley since I'm not back in the office until a week from Friday (wedding, PuppetConf, a grow class), could you show @slevenick how to upload + test pipelines to Concourse? I didn't get a chance to before leaving for AnsibleFest |
…modules into chef-bigquery
30013cf
to
d1da2b1
Compare
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit 3cd6a17) have been included in your existing downstream PRs. |
Oh sure, sure, sure! Come grab me whenever. |
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit 9cd4fe7) have been included in your existing downstream PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I merged master to pick up the Travis fix so things should build now.
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit c398c84) have been included in your existing downstream PRs. |
I am (still) a robot that works on MagicModules PRs! I just wanted to let you know that your changes (as of commit a5e7931) have been included in your existing downstream PRs. |
[all]
[terraform]
[puppet]
[puppet-bigquery]
[puppet-compute]
[puppet-container]
[puppet-dns]
[puppet-logging]
[puppet-pubsub]
[puppet-resourcemanager]
[puppet-sql]
[puppet-storage]
[chef]
Add BigQuery product
[chef-compute]
[chef-container]
[chef-dns]
[chef-logging]
[chef-spanner]
[chef-sql]
[chef-storage]
[ansible]